-
Notifications
You must be signed in to change notification settings - Fork 222
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update files generated from the k6 Extension Registry #1751
Conversation
|
a344303
to
5537e4b
Compare
@szkiba something doesn't look right here. 🤔 Do you know what the script was trying to do? |
Ooops, I just accepted some last minute change proposal on PR from @jdbaldry and it seem some of them has a syntax problem... |
5537e4b
to
48849bd
Compare
I re-ran the workflow manually in https://github.com/grafana/k6-docs/actions/runs/11111741710 |
Much better now :) |
@@ -51,10 +47,6 @@ Use the table to explore the many extensions. Questions? Feel free to join the d | |||
<h4>xk6-cache</h4> | |||
<p>Enable vendoring remote HTTP modules to a single source-control-friendly file</p> | |||
</a> | |||
<a href="https://github.com/fornfrey/xk6-celery" target="_blank" class="nav-cards__item nav-cards__item--guide"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The extension does not meet the registry requirements.
fornfrey/xk6-celery#7
fornfrey/xk6-celery#6
@@ -35,10 +35,6 @@ Use the table to explore the many extensions. Questions? Feel free to join the d | |||
<h4>xk6-ansible-vault</h4> | |||
<p>Encrypt and decrypt Ansible Vault</p> | |||
</a> | |||
<a href="https://github.com/joanlopez/xk6-aws" target="_blank" class="nav-cards__item nav-cards__item--guide"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The extension does not meet the registry requirements.
joanlopez/xk6-aws#2
joanlopez/xk6-aws#1
@@ -95,14 +87,6 @@ Use the table to explore the many extensions. Questions? Feel free to join the d | |||
<h4>xk6-dotenv</h4> | |||
<p>Load env vars from a .env file</p> | |||
</a> | |||
<a href="https://github.com/szkiba/xk6-ts" target="_blank" class="nav-cards__item nav-cards__item--guide"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Extension owner changed this is why name changed from "szkiba/xk6-ts" to "grafana/xk6-ts"
@@ -323,6 +275,10 @@ Use the table to explore the many extensions. Questions? Feel free to join the d | |||
<h4>xk6-tracetest</h4> | |||
<p>Support for Tracetest test execution and tracing client</p> | |||
</a> | |||
<a href="https://github.com/grafana/xk6-ts" target="_blank" class="nav-cards__item nav-cards__item--guide"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Extension owner changed this is why name changed from "szkiba/xk6-ts" to "grafana/xk6-ts"
<h4>xk6-ts</h4> | ||
<p>Add TypeScript support for k6</p> | ||
</a> | ||
<a href="https://github.com/BarthV/xk6-es" target="_blank" class="nav-cards__item nav-cards__item--guide"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The extension does not meet the registry requirements
BarthV/xk6-es#7
BarthV/xk6-es#6
@@ -135,14 +119,6 @@ Use the table to explore the many extensions. Questions? Feel free to join the d | |||
<h4>xk6-git</h4> | |||
<p>Clone Git repositories from tests</p> | |||
</a> | |||
<a href="https://github.com/AckeeCZ/xk6-google-iap" target="_blank" class="nav-cards__item nav-cards__item--guide"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The extension does not meet the registry requirements
AckeeCZ/xk6-google-iap#10
AckeeCZ/xk6-google-iap#9
AckeeCZ/xk6-google-iap#8
<h4>xk6-google-iap</h4> | ||
<p>Provides access to Google Auth token</p> | ||
</a> | ||
<a href="https://github.com/heww/xk6-harbor" target="_blank" class="nav-cards__item nav-cards__item--guide"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The extension does not meet the registry requirements
goharbor/xk6-harbor#16
goharbor/xk6-harbor#15
@@ -185,15 +161,7 @@ Use the table to explore the many extensions. Questions? Feel free to join the d | |||
</a> | |||
<a href="https://github.com/pmalhaire/xk6-mqtt" target="_blank" class="nav-cards__item nav-cards__item--guide"> | |||
<h4>xk6-mqtt</h4> | |||
<p>mqtt extension</p> | |||
</a> | |||
<a href="https://github.com/ydarias/xk6-nats" target="_blank" class="nav-cards__item nav-cards__item--guide"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The extension does not meet the registry requirements
ydarias/xk6-nats#15
ydarias/xk6-nats#14
<h4>xk6-nats</h4> | ||
<p>Provides NATS support</p> | ||
</a> | ||
<a href="https://github.com/patrick-janeiro/xk6-neo4j" target="_blank" class="nav-cards__item nav-cards__item--guide"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The extension does not meet the registry requirements
patrick-janeiro/xk6-neo4j#5
<a href="https://github.com/patrick-janeiro/xk6-neo4j" target="_blank" class="nav-cards__item nav-cards__item--guide"> | ||
<h4>xk6-neo4j</h4> | ||
<p>Interact with Neo4J graph databases</p> | ||
<p>MQTT extension</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
MQTT is an abbreviation
@@ -215,10 +183,6 @@ Use the table to explore the many extensions. Questions? Feel free to join the d | |||
<h4>xk6-output-clickhouse</h4> | |||
<p>Export results to ClickHouse</p> | |||
</a> | |||
<a href="https://github.com/dynatrace/xk6-output-dynatrace" target="_blank" class="nav-cards__item nav-cards__item--guide"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The extension does not meet the registry requirements
Dynatrace/xk6-output-dynatrace#15
Dynatrace/xk6-output-dynatrace#14
Dynatrace/xk6-output-dynatrace#13
@@ -255,14 +219,10 @@ Use the table to explore the many extensions. Questions? Feel free to join the d | |||
<h4>xk6-output-timestream</h4> | |||
<p>Export results to AWS Timestream</p> | |||
</a> | |||
<a href="https://github.com/wosp-io/xk6-playwright" target="_blank" class="nav-cards__item nav-cards__item--guide"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
owner changed
@@ -255,14 +219,10 @@ Use the table to explore the many extensions. Questions? Feel free to join the d | |||
<h4>xk6-output-timestream</h4> | |||
<p>Export results to AWS Timestream</p> | |||
</a> | |||
<a href="https://github.com/wosp-io/xk6-playwright" target="_blank" class="nav-cards__item nav-cards__item--guide"> | |||
<a href="https://github.com/nicholasvuono/xk6-playwright" target="_blank" class="nav-cards__item nav-cards__item--guide"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
owner changed
<h4>xk6-playwright</h4> | ||
<p>Browser automation and end-to-end web testing using Playwright</p> | ||
</a> | ||
<a href="https://github.com/mcosta74/xk6-plist" target="_blank" class="nav-cards__item nav-cards__item--guide"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The extension does not meet the registry requirements
mcosta74/xk6-plist#2
@@ -287,10 +247,6 @@ Use the table to explore the many extensions. Questions? Feel free to join the d | |||
<h4>xk6-sql</h4> | |||
<p>Load-test SQL Servers</p> | |||
</a> | |||
<a href="https://github.com/mridehalgh/xk6-sqs" target="_blank" class="nav-cards__item nav-cards__item--guide"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The extension does not meet the registry requirements
mridehalgh/xk6-sqs#8
@@ -303,11 +259,7 @@ Use the table to explore the many extensions. Questions? Feel free to join the d | |||
<h4>xk6-stomp</h4> | |||
<p>Client for STOMP protocol</p> | |||
</a> | |||
<a href="https://github.com/NAlexandrov/xk6-tcp" target="_blank" class="nav-cards__item nav-cards__item--guide"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The extension does not meet the registry requirements
NAlexandrov/xk6-tcp#7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@heitortsergent , I reviewed all the changes and wrote a justification for them.
Would you please look it over and if you consider the justifications appropriate, then approve it.
In most cases, the extension does not meet the requirements of the registry. I opened issues about these in the extensions repository about 2 months ago and received no response. They don't seem to be maintained.
@dgzlopes I added you as a reviewer here. @szkiba this is related to the extensions registry project, @szkiba implemented a way for us to keep the information between the extensions registry and the docs in sync. There are a few extensions that would be removed from our docs list because they don't meet our extensions requirements. I'm wondering, is it okay to remove all of the ones listed in this PR, or are there any of them that might still be useful for users, even if they don't meet all of our requirements? 🤔 |
48849bd
to
606f6e0
Compare
606f6e0
to
85c4c7d
Compare
@heitortsergent, I see that Ivan reached out to all Extension developers asking them to update their extensions... To be honest, I'm a bit sad to see a few of these go away (e.g., Harbor), but unless we fork them (& we don't have resources to maintain more stuff), I'm not sure there is anything that can be done. |
Ok, I guess I will merge this for now. I haven't kept up with the conversations around the extensions registry, but maybe there's some way for us to have a list of useful extensions that might not be "recommended" somewhere? 🤔 |
The k6 Extension Registry has changed.
This pull request contains the files generated from the k6 Extension Registry.